Skip to content

Update Paths.php#4093

Closed
abdounikarim wants to merge 2 commits into
api-platform:mainfrom
abdounikarim:patch-1
Closed

Update Paths.php#4093
abdounikarim wants to merge 2 commits into
api-platform:mainfrom
abdounikarim:patch-1

Conversation

@abdounikarim

Copy link
Copy Markdown
Contributor
Q A
Branch? main for features / current stable version branch for bug fixes
Bug fix? no
New feature? no
Deprecations? no
Tickets Fix #...
License MIT
Doc PR api-platform/docs#...

Hello, i just add a method if we want to remove one or many paths from the docs.
correcting php-cs-fixer
@alanpoulain

Copy link
Copy Markdown
Member

Hello, thanks for the PR. Could you add more context to your PR? Like describing a use case?

@abdounikarim

Copy link
Copy Markdown
Contributor Author

Hello @alanpoulain , yep, of course.
I think we can have an API in production and we don't want to give access to all routes (like hiding some routes which are for example still in dev or test mode), or if we want to give access only on a part of an API to users.

Does that answer to your question ? Cheers

@alanpoulain

Copy link
Copy Markdown
Member

Thanks. @soyuka could you look at it?

@soyuka

soyuka commented Mar 1, 2021

Copy link
Copy Markdown
Member

Sorry this doesn't fit the Immutable model of these classes, what about #4099 to remove an operation?

@abdounikarim

Copy link
Copy Markdown
Contributor Author

@soyuka we add more code 😛

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants